Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Modify Pad Conv ConvTranspose Slice Add RandomNormal Tile ops #27

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

GuardSkill
Copy link

@GuardSkill GuardSkill commented Sep 16, 2021

  1. Modify Conv op support F.Conv()
  2. Modify Pad op for general support
  3. Modify Slice op support negative step
    3 .Add Tile and RandomNormal ops

1 Conv  op support F.Conv()
2 Slice op support negative step
3 add Tile and RandomNormal ops
@fumihwh
Copy link
Owner

fumihwh commented Sep 18, 2021

@GuardSkill
Awesome! Thanks for contribution!
Please use yapf to reformat code by using style file .style.yapf.

@fumihwh
Copy link
Owner

fumihwh commented Sep 19, 2021

FYI,following test failed.

=========================== short test summary info ============================
FAILED onnx_pytorch/tests/test_base.py::TestBase::test_pad - assert False
FAILED onnx_pytorch/tests/test_base.py::TestBase::test_pad_5D - assert False
FAILED onnx_pytorch/tests/test_base.py::TestBase::test_pad_reflect - Assertio...
FAILED onnx_pytorch/tests/test_onnx_model_zoo.py::TestModel::test_vision_style_transfer_fast_neural_style_candy
FAILED onnx_pytorch/tests/test_onnx_model_zoo.py::TestModel::test_vision_style_transfer_fast_neural_style_mosaic
FAILED onnx_pytorch/tests/test_onnx_model_zoo.py::TestModel::test_vision_style_transfer_fast_neural_style_pointilism
FAILED onnx_pytorch/tests/test_onnx_model_zoo.py::TestModel::test_vision_style_transfer_fast_neural_style_rain_princess
FAILED onnx_pytorch/tests/test_onnx_model_zoo.py::TestModel::test_vision_style_transfer_fast_neural_style_udnie
======= 8 failed, 97 passed, 1 skipped, 4 warnings in 221.06s (0:03:41) ========

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants